Skip to content

Add pedestal transport model section to CombinedTransportModel#1330

Merged
copybara-service[bot] merged 1 commit intomainfrom
pedestal-transport
Jul 25, 2025
Merged

Add pedestal transport model section to CombinedTransportModel#1330
copybara-service[bot] merged 1 commit intomainfrom
pedestal-transport

Conversation

@theo-brown
Copy link
Collaborator

@theo-brown theo-brown commented Jun 25, 2025

  • Add separate pedestal model to CombinedTransportModel
  • Update tests
  • Update docs
  • Add test to check behaviour when no pedestal model supplied - intended behaviour is revert to chi_min, etc

Implements #941

@theo-brown theo-brown force-pushed the pedestal-transport branch from 43b78a4 to 9653014 Compare June 25, 2025 16:25
@theo-brown theo-brown force-pushed the pedestal-transport branch 2 times, most recently from 53be1e0 to 48c4311 Compare June 26, 2025 15:04
@theo-brown theo-brown requested a review from jcitrin June 26, 2025 15:08
Copy link
Collaborator

@jcitrin jcitrin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In addition to the comments, please update the docs following the converged-upon patterns.

@theo-brown theo-brown force-pushed the pedestal-transport branch 3 times, most recently from a2fa146 to 6636467 Compare July 4, 2025 10:54
@theo-brown theo-brown requested a review from tamaranorman July 4, 2025 10:58
@theo-brown theo-brown requested a review from jcitrin July 7, 2025 09:05
@theo-brown theo-brown self-assigned this Jul 17, 2025
Copy link
Collaborator

@jcitrin jcitrin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Following the small fixes + fixing the merge conflicts, will bring it it

@theo-brown theo-brown force-pushed the pedestal-transport branch 2 times, most recently from 477dce7 to e73a1e9 Compare July 25, 2025 08:36
@theo-brown theo-brown force-pushed the pedestal-transport branch from e73a1e9 to 7efed82 Compare July 25, 2025 08:37
@theo-brown
Copy link
Collaborator Author

I've made the changes, rebased onto main, and squashed into a single commit, let me know if there's anything more that needs doing!

@theo-brown theo-brown requested a review from jcitrin July 25, 2025 08:51
@jcitrin jcitrin added the copybara:import-manual Set when ready for copybara manual import label Jul 25, 2025
@copybara-service copybara-service bot merged commit c9ab82e into main Jul 25, 2025
21 checks passed
@theo-brown theo-brown deleted the pedestal-transport branch July 26, 2025 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

copybara:import-manual Set when ready for copybara manual import

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add pedestal region transport coefficient values to the pedestal config, instead of using the minimum values

3 participants